Skip to content

Handle tarballs with dev.eessi.io project subdir in name#212

Merged
bedroge merged 6 commits intoEESSI:mainfrom
Neves-P:fork/main
Apr 17, 2025
Merged

Handle tarballs with dev.eessi.io project subdir in name#212
bedroge merged 6 commits intoEESSI:mainfrom
Neves-P:fork/main

Conversation

@Neves-P
Copy link
Copy Markdown
Member

@Neves-P Neves-P commented Apr 15, 2025

The software-layer PR (link incoming) for support for dev.eessi.io builds encodes the subdir name in the tarball. Because the file tree has a different depth and the position of the project subdirectory in the tree moved, this also had to be adjusted. This PR handles these changes and should not impact other builds for software.eessi.io.

Comment thread scripts/ingest-tarball.sh Outdated
Comment on lines +261 to +262
# Handle longer prefix with project name in dev.eessi.io and
# get
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and get... the right basedir? 😄

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦 😅 , I didn't push the latest commit

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in badfbf8

@Neves-P
Copy link
Copy Markdown
Member Author

Neves-P commented Apr 15, 2025

EESSI/dev.eessi.io-example#23

One tarball was created, waiting for two more. We can test we just one, if so, adding the bot: deploy label should be enough 👍

Comment thread scripts/ingest-tarball.sh
Co-authored-by: Bob Dröge <b.e.droge@rug.nl>
@bedroge bedroge merged commit 54ed29e into EESSI:main Apr 17, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants